Skip to content

style: apply ruff format and fix the trailing whitespace findings - #13

Merged
Jeomon merged 1 commit into
Jeomon:mainfrom
hussainbiedouh:style/apply-ruff-format
Oct 27, 2026
Merged

Jeomon merged 1 commit into
Jeomon:mainfrom
hussainbiedouh:style/apply-ruff-format

Conversation

@hussainbiedouh

Copy link
Copy Markdown
Contributor

The lint job runs pre-commit run --all-files, and ruff 0.12.1 currently reports 3 W293 findings and wants to reformat 7 files:

Found 3 errors.
[*] 3 fixable with the `--fix` option.
Would reformat: windows_use\agent\service.py
Would reformat: windows_use\providers\azure_openai\llm.py
...
7 files would be reformatted, 120 files already formatted

The findings all come from commits that landed after the last "Apply ruff-format to entire codebase" pass.

This is a reformatting only: line wrapping, quote style, and one import reorder inside from typing import ... that pulls in the same names either way. The abstract syntax tree of every touched file is unchanged apart from that import order.

After this, pre-commit run --all-files passes locally:

ruff (legacy alias)......................................................Passed
ruff format..............................................................Passed
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...............................................................Passed
check for added large files..............................................Passed
debug statements (python)................................................Passed

The lint job runs the pre-commit hooks, and ruff 0.12.1 reports 3 W293
findings and wants to reformat 7 files. The changes all come from commits
that landed after the last "Apply ruff-format to entire codebase" pass.

Reformatting only: line wrapping, quote style and one import reorder inside
`from typing import ...`, which pulls in the same names either way. The
abstract syntax tree of every touched file is unchanged apart from that
import order.

With this, `pre-commit run --all-files` passes.
@Jeomon
Jeomon merged commit 1be0c21 into Jeomon:main Oct 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants